Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

Use quay.io/kinvolk-ci/registry in Tinkerbel sandbox #1454

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

meyskens
Copy link
Contributor

@meyskens meyskens commented Mar 29, 2021

Use quay.io/kinvolk-ci/registry in Tinkerbel sandbox

This is needed to bypass the Docker Hub ratelimit which we hit on the CI. Other images like Postgres and Ubuntu seem to be whitelisted as library images against the rate limit at the moment.

How to use

Check if the CI passes

This is needed to bypass the Docker Hub ratelimit which we hit on the CI
 pipeline.

Signed-off-by: Maartje Eyskens <maartje@eyskens.me>
@meyskens meyskens force-pushed the meyskens/fix-tinkerbell branch from fe62a70 to 5493440 Compare March 29, 2021 09:56
@meyskens meyskens requested review from iaguis and surajssd March 30, 2021 07:58
@invidian
Copy link
Member

invidian commented Apr 1, 2021

See #1243. We also hit limits for other images 😞

@surajssd
Copy link
Member

surajssd commented Apr 1, 2021

Having a way to update images similar to calico can be helpful. That way the folks doing updates later can just run script and update stuff in one stroke.

@invidian
Copy link
Member

invidian commented Apr 1, 2021

Having a way to update images similar to calico can be helpful. That way the folks doing updates later can just run script and update stuff in one stroke.

Yes, but also I'd avoid touching the Tinkerbell upstream scripts as much as possible. We should try to get such changes upstream first.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants